Skip to content

Improve OAuth token URL configuration errors - #2245

Draft
lissam1 wants to merge 2 commits into
5.3.xfrom
fix-oauth-logs
Draft

Improve OAuth token URL configuration errors#2245
lissam1 wants to merge 2 commits into
5.3.xfrom
fix-oauth-logs

Conversation

@lissam1

@lissam1 lissam1 commented Jul 8, 2026

Copy link
Copy Markdown
Member

Fixes #2179

Adds validation for configured OAuth token endpoint URLs so invalid values fail with a clear ConfigurationException.
Changes:

  • Validate micronaut.security.oauth2.clients.*.token.url when building the token endpoint.
  • Require an absolute http or https URL with a valid host.
  • Include the OAuth client name, property path, and configured value in the error message.
  • Add regression coverage for missing schemes, malformed HTTP URLs, relative paths, and unsupported schemes.

@lissam1
lissam1 marked this pull request as ready for review July 8, 2026 17:37
@lissam1
lissam1 requested a review from sdelamo July 8, 2026 17:37
@lissam1 lissam1 added the type: improvement A minor improvement to an existing feature label Jul 8, 2026
@github-project-automation github-project-automation Bot moved this to Backlog in 5.1.0 Release Jul 8, 2026
@lissam1 lissam1 moved this from Backlog to In review in 5.1.0 Release Jul 13, 2026
@sdelamo
sdelamo marked this pull request as draft July 13, 2026 13:02

@sdelamo sdelamo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should do this with an annotation constraint.

@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: improvement A minor improvement to an existing feature

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

Improve Error messages on OAuth verification

2 participants